New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cleanup exchange #528

Merged

tiero merged 3 commits into tdex-network:master from Janaka-Steph:refacto-exchange

Mar 11, 2022

Collaborator

Janaka-Steph commented Mar 10, 2022 •

edited

Loading

Simplify updateReceiveSats and updateSendSats in src/components/TdexOrderInput/hooks.ts
Use IonAlert as a component to have simpler code and fix edge cases
Hide "Market provided by: ..." if sendSats is zero
New action replaceMarketsOfProvider instead of addMarkets to update properly when provider/markets goes down. Also simpler code
Log createDiscoverer errors
Fix getMarketsFromProvider retry (not catching in fn)

Please review @tiero

Janaka-Steph added 3 commits

March 10, 2022 21:10


          refactor

124fbbb


          use debug to log createDiscoverer errors

186f6bf


          Fix getMarketsFromProvider retry

tiero merged commit e47f388 into tdex-network:master

Janaka-Steph deleted the refacto-exchange branch

March 15, 2022 19:21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet